Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update MMK for OpenVMS x86_64 #101

Merged
merged 9 commits into from
Sep 13, 2023
Merged

Conversation

craigberry
Copy link
Contributor

No description provided.

craigberry and others added 9 commits February 25, 2023 07:29
As the list of architectures now grows to include x86_64, it's
easier and more robust to check for anything but VAX since all
the later architectures behave the same for the things being
ifdef'd.
Rather than adding a fourth hard-coded option to the list, let's
derive the built-in name from the architecture name we already
have.  Some of them come back from sys$getsyi in mixed case, so
upper case the result when defining the built-in.

N.B.  The existing behavior differs from MMS and is not changed
here.  MMS will set, for example, MMSALPHA to a value of "ALPHA"
whereas MMK sets it to "1".  Either should work in an IFDEF, so
it may not matter.
Otherwise we get:

$ MACRO/MIGRATION/NOLIST/OBJECT=BIN_DIR:PARSE_TABLES.OBJ -
	SYS$LIBRARY:ARCH_DEFS.MAR+SYS$DISK:[]PARSE_TABLES.MAR

EVAX	=	1
^
%XMAC-W-GENWARN, generated WARNING: 0 Architecture symbol set to 1, probably an error
This rounds out some things that were missing, such as kitinstal
updates.
MMK now propely handles DCL command lines longer than 255 characters on VAX.
Such lines are broken up into continuation lines that are passed to the
worker subprocess. The code was there in previous versions, but the limit
was defined incorrectly, so the continuation lines were 257 characters long,
which was two characters too long for VAX.
@tesneddon
Copy link
Member

Thanks @craigberry and @huntergoatley for all your help with this.

Pressing the button...

@tesneddon tesneddon merged commit 84cb968 into endlesssoftware:master Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants